Skip to content

Conversation

@wling-art
Copy link
Contributor

@wling-art wling-art commented Dec 14, 2025

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 14, 2025

Deploying sapling with  Cloudflare Pages  Cloudflare Pages

Latest commit: f8ea0c6
Status:⚡️  Build in progress...

View logs

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR implements an interactive file tree explorer component to replace static text-based directory structures throughout the documentation. The implementation is inspired by PaperMC's documentation and provides a more user-friendly way to visualize file and folder hierarchies with hover-over descriptions.

Key changes:

  • Added a reusable StructureExplorer React component with popup descriptions
  • Created 10 data structure files covering various Minecraft server configurations
  • Converted multiple .md files to .mdx to support React components
  • Added @iconify/react dependency for folder/file icons
  • Implemented dark/light theme support for the component

Reviewed changes

Copilot reviewed 28 out of 30 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
tsconfig.json Added baseUrl configuration to support path aliases for imports
src/types/ExplorerNode.ts Defined TypeScript interface for file tree node structure
src/data/structures/*.ts Created 10 data structure files (viaproxy, server directories, BDS, Nukkit, Maven, Gradle, etc.)
src/data/structures/index.ts Barrel export file for all structure data
src/css/structure_explorer.css Component styling with tree lines, hover effects, and popup windows
src/css/base/theme.css Added CSS variables for dark/light theme support
src/components/StructureExplorer.tsx Main React component implementing the interactive file tree
package.json Added @iconify/react dependency
pnpm-lock.yaml Lock file updates for new dependency
docs/*.mdx Converted 7 documentation files from .md to .mdx and integrated the component
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

docs-java/start/basic/server-dir-structure.mdx:7

  • The import statement uses a direct path to the data file instead of importing from the barrel export in the index. This should use import { serverDirStructureData } from "@site/src/data/structures" to maintain consistency with other imports and make refactoring easier.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Dreeam-qwq Dreeam-qwq added the type: docs site Pull request for code changes related to the site itself label Dec 14, 2025
Copy link
Contributor

@Dreeam-qwq Dreeam-qwq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

看起来不戳

@wling-art wling-art marked this pull request as draft December 15, 2025 08:31
@wling-art wling-art marked this pull request as ready for review December 15, 2025 08:37
@wling-art wling-art merged commit 22873f4 into main Dec 15, 2025
2 of 3 checks passed
@wling-art wling-art deleted the feat/filetree2 branch December 15, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: docs site Pull request for code changes related to the site itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants